Skip to content

Conversation

msrd0
Copy link
Collaborator

@msrd0 msrd0 commented Jan 23, 2025

https://blog.rust-lang.org/2025/01/09/Rust-1.84.0.html#cargo-considers-rust-versions-for-dependency-version-selection

Our MSRV is currently 1.65, otherwise we cannot find a candidate for the regex-syntax crate which is a transitive dependency.

This PR also updates the version of clippy we use to 1.88, and resolves all warnings introduced in the newer versions.

@msrd0 msrd0 marked this pull request as ready for review January 23, 2025 23:32
Empty2k12
Empty2k12 previously approved these changes Jan 24, 2025
Copy link
Collaborator

@Empty2k12 Empty2k12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does indeed look like a better solution. Thanks for figuring and also building it out!

I assume I thought this was necessary for the readme generation, but
turns out the readme looks just fine.
@msrd0 msrd0 requested a review from Empty2k12 August 20, 2025 10:24
Copy link
Collaborator

@Empty2k12 Empty2k12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks for tackling this!

@Empty2k12 Empty2k12 merged commit b3598e8 into main Aug 21, 2025
22 checks passed
@Empty2k12 Empty2k12 deleted the ci-msrv-1.84 branch August 21, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants